Skip to content

[test] 테스트 코드 수정 (User.SetId() 제거) #121

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 2 commits into from
Jul 25, 2025
Merged

[test] 테스트 코드 수정 (User.SetId() 제거) #121

merged 2 commits into from
Jul 25, 2025

Conversation

@sehee123
Copy link
Collaborator

@sehee123 sehee123 commented Jul 25, 2025 •
edited
Loading

🛰️ Issue Number

🪐 작업 내용

  • dbRider를 적용해보려고했으나, 스레드 동시성 테스트 같은 경우 스레드갯수에 따라서 동적으로 user를 생성하고있기에 적용하지 못함
  • User.setId()를 제거하기위해 리플렉션 사용
  • sessionService 테스트에서는 @ExtendWith(MockitoExtension.class) 로 단위테스트를 하고있기 때문에 dbRider를 적용하지 않았고
  • 위와 같이 리플렉션으로 setId를 대체하였습니다!

📚 Reference

✅ Check List

  • 코드가 정상적으로 컴파일되나요?
  • 테스트 코드를 통과했나요?
  • merge할 브랜치의 위치를 확인했나요?
  • Label을 지정했나요?

@sehee123 sehee123 self-assigned this Jul 25, 2025
@sehee123 sehee123 added the test 테스트 코드 작성 label Jul 25, 2025
@sehee123 sehee123 linked an issue Jul 25, 2025 that may be closed by this pull request
[test] test 코드 리팩토링 #117
Closed
2 tasks
@sehee123 sehee123 merged commit 407f4f2 into dev Jul 25, 2025
3 of 5 checks passed
@sehee123 sehee123 deleted the test/117 branch July 25, 2025 09:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@dlsrks1021 dlsrks1021 dlsrks1021 approved these changes

@jiwon1217 jiwon1217 jiwon1217 approved these changes

@silver-eunjoo silver-eunjoo silver-eunjoo approved these changes

Assignees

@sehee123 sehee123

Labels

test 테스트 코드 작성

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[test] test 코드 리팩토링

5 participants

@sehee123 @dlsrks1021 @jiwon1217 @silver-eunjoo